Codecov Report

❌ Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 82.94%. Comparing base (4c0072f) to head (c67f41b).
⚠️ Report is 59 commits behind head on main.

@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
+ Coverage   82.88%   82.94%   +0.06%     
==========================================
  Files          46       46              
  Lines        4212     4217       +5     
  Branches     1040     1041       +1     
==========================================
+ Hits         3491     3498       +7     
- Misses        692      694       +2     
+ Partials       29       25       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

integrations test will fail until KERIA PR is merged

@rodolfomiranda There is something wrong after the upgrade to 0.2.0-dev0. I tried to debug the multisig-vlei test and found that after the client of GAR1 resolves the oobi for the multisig QVI aid, they still don't have the QVI aid in the contact list.

To get this merged we need to figure out what the problem is there. Perhaps it would be better to create a separate PR for just the upgrade to 0.2.0-dev0.

@rodolfomiranda Did a little bit more digging, and what we get back from the long running operation for resolving the QVI AID is just ({ oobi: "oobi url" }), rather than the expected response of the current state of the contact. This only happens here:

https://github.com/WebOfTrust/keria/blob/7ee5ba84c9aed5d2d21fa48d8241d39b06aadb93/src/keria/core/longrunning.py#L239, which only happens if the obr.cid is not in kevers. Any ideas?

Any ideas?

I've been debugging a lot and saw the same but also found that the problem starts to happens after a new ixn or rot event is applied to the multisig. In the multisig-vlei-issuanse test, that happens after the ixn for the delegation in GEDA aid; in the multisig test that happens after rotating the multisig aid. In both cases, if you comment out the event (the delegation in the vlei test or the rotation in the multisig test), the problem not longer appears. If not, seems that the habs got corrupted.

I think it's related to the changes that were applied in keripy and keria to track multisig states, such as the smids and rmids as states. It works well at inception, but something is wrong after and ixn or rot event. I'm still troubleshooting...

Any ideas?

I think it's related to the changes that were applied in keripy and keria to track multisig states, such as the smids and rmids as states. It works well at inception, but something is wrong after and ixn or rot event. I'm still troubleshooting...

Cross posting for visibility.

There are probably more issues then. In #252, I managed to reproduce a similar situtation for single sig only, where the problem is that a delegator cannot resolve an oobi of the delegatee.

@rodolfomiranda I believe this has been superseded by a bunch of other merged PRs now. Should we close this one?

@rodolfomiranda and @lenkan has the work in this PR been added through other PRs? It's been just about a year since we looked at this and we should probably close this PR and open a new one.

has the work in this PR been added through other PRs?

I believe the method for renaming AIDs is already added. But I haven't verified. I think you can close it.

There is for renaming, but not deleting as I believe there was a discussion on deletion vs abandoning (rotating to empty keys) and it was never completed. But this PR is extremely old, it should be closed.